Skip to content

#127 - Remove LHS navigation from Holoscan integration tutorial#201

Merged
RamyaGuru merged 2 commits into
mainfrom
127-remove-holoscan-tutorial-lhs-nav
Jun 22, 2026
Merged

#127 - Remove LHS navigation from Holoscan integration tutorial#201
RamyaGuru merged 2 commits into
mainfrom
127-remove-holoscan-tutorial-lhs-nav

Conversation

@RamyaGuru

Copy link
Copy Markdown
Collaborator

Summary

The DAQIRI + Holoscan Integration tutorial was the only tutorial page rendering a left-hand navigation menu. All sibling tutorial pages (system_configuration.md, configuration-walkthrough.md, bare-metal-cmake-build.md) use hide: [navigation] frontmatter. This adds the same frontmatter to daqiri-holoscan-integration.md so the LHS menu is gone and the section behaves uniformly.

Also adds a check for this class of drift to the docs-sync rule (.claude/rules/docs-sync.md), under "Landing page and navigation" and the mapping quick reference, since it fell outside the rule's existing scope.

Refs #127

Test plan

  • Verified locally with mkdocs serve — the Holoscan tutorial page no longer shows the LHS navigation menu and matches the other tutorial pages. Build is clean.

🤖 Generated with Claude Code

RamyaGuru and others added 2 commits June 22, 2026 10:17
Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
Signed-off-by: Ramya Gurunathan <rgurunathan@nvidia.com>
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Signed-off-by: Ramya Gurunathan <rgurunathan@nvidia.com>
@greptile-apps

greptile-apps Bot commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

Adds hide: [navigation] frontmatter to docs/tutorials/daqiri-holoscan-integration.md so the Holoscan tutorial no longer renders the LHS menu, matching the three sibling tutorial pages. A companion note is added to .claude/rules/docs-sync.md to prevent the same drift on future tutorial additions.

  • daqiri-holoscan-integration.md now opens with the same three-line frontmatter block (---\nhide:\n - navigation\n---) used by system_configuration.md, configuration-walkthrough.md, and bare-metal-cmake-build.md.
  • .claude/rules/docs-sync.md gains a "Page frontmatter consistency" bullet and a new quick-reference row covering new docs/tutorials/*.md pages.

Confidence Score: 5/5

Safe to merge — the change is a two-line docs frontmatter addition with no code or build-system impact.

Both commits carry DCO sign-offs and the correct #127 - ... title format. The frontmatter added to the Holoscan tutorial is byte-for-byte identical to what the three sibling pages already use, and the docs-sync rule update accurately captures the new check without altering existing entries.

No files require special attention.

Important Files Changed

Filename Overview
docs/tutorials/daqiri-holoscan-integration.md Adds hide: [navigation] frontmatter to match all three sibling tutorial pages — correct fix for the LHS nav inconsistency.
.claude/rules/docs-sync.md Adds a new frontmatter-consistency note under "Landing page and navigation" and a corresponding quick-reference table row — no existing rules altered.

Reviews (1): Last reviewed commit: "#127 - Add tutorial frontmatter consiste..." | Re-trigger Greptile

@RamyaGuru RamyaGuru merged commit 3cce706 into main Jun 22, 2026
3 checks passed
@RamyaGuru RamyaGuru deleted the 127-remove-holoscan-tutorial-lhs-nav branch June 22, 2026 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant